Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

dagger environment #18

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

dagger environment #18

wants to merge 11 commits into from

Conversation

garbas
Copy link
Contributor

@garbas garbas commented Oct 18, 2024 •
edited
Loading

The environment adds a package from a flake that dagger is keeping up to date.

It also adds a service to run a Dagger Engine, if any configuration is needed.

@floxbot floxbot added the team-content Issues to be picked up by @flox/team-content label Oct 18, 2024
@garbas garbas marked this pull request as draft October 18, 2024 19:47
@garbas garbas changed the title dagger init dagger environment Oct 18, 2024
@garbas garbas requested a review from stahnma October 21, 2024 14:52
@garbas garbas marked this pull request as ready for review October 21, 2024 14:52
@garbas
Copy link
Contributor Author

garbas commented Oct 21, 2024

This is blocked until we get docker installed on runners (ping @bryanhonof )

@stahnma stahnma added the team-developer-support Issues to be picked up by @flox/team-developer-support label Jan 21, 2025
@bryanhonof
Copy link
Member

bryanhonof commented Jan 23, 2025 •
edited
Loading

The macOS builders run Colima under a user colima, Colima can't mount system folders or other user's folders, and one of the first things we're doing in our CI is ssh github@... and mktemp -d, running all other commands from a temporary directory. The Flox env has a service definition that expects to be able to do --volume "$PWD:/somewhere", this doesn't actually work because Colima has no access to mount the /tmp directory. Looking into it, but that's why the CI is failing.

@garbas
Copy link
Contributor Author

garbas commented Jan 24, 2025

@bryanhonof thank you for looking into this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stahnma stahnma Awaiting requested review from stahnma

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
team-content Issues to be picked up by @flox/team-content team-developer-support Issues to be picked up by @flox/team-developer-support
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@garbas @bryanhonof @stahnma @floxbot